-
Notifications
You must be signed in to change notification settings - Fork 145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
copy_object: support passing another request options #513
Conversation
For example: passing `destination_predefined_acl`
Ah, thank you so much! Esp. for including tests ^__^ I'll merge once integration tests pass. |
Codecov Report
@@ Coverage Diff @@
## master #513 +/- ##
==========================================
- Coverage 88.49% 88.46% -0.04%
==========================================
Files 311 311
Lines 5217 5217
==========================================
- Hits 4617 4615 -2
- Misses 600 602 +2
Continue to review full report at Codecov.
|
@yosiat Do you want me to release the changes right away? |
@Temikus if that's possible it will be awesome! since I need to do another PR in carrierwave. |
@yosiat done! |
For example: passing
destination_predefined_acl
handles: #505 and #499 (comment)